Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid hotcue buttons from being stucked in the pressed state #3344

Merged
merged 4 commits into from
Nov 28, 2020

Conversation

daschuer
Copy link
Member

We have and issue when someone pressed right click during preview or outside the button.
In this case Mixxx sticks in the preview state, the user looses control which must not happen in any case.
This PR aims to fix that.

@ronso0
Copy link
Member

ronso0 commented Nov 23, 2020

I'm afraid this only partly solves the issue because there are still two ways to get stuck in preview mode:

  • press & hold a GUI hotcue button to preview, then press the same hotcue button on the keyboard
  • press & hold the Cue button > move outside > release

In both cases the initial GUI button is released but Mixxx is still in preview mode.
Guess this requires a fix in cuecontrol.cpp

@daschuer
Copy link
Member Author

Which skin did you use for testing?

@ronso0
Copy link
Member

ronso0 commented Nov 23, 2020

I tested with LateNight, PaleMoon

@ronso0
Copy link
Member

ronso0 commented Nov 23, 2020

..and now Shade. Same result unfortunately.

@daschuer
Copy link
Member Author

I was able to reproduce it when using the hotcue buttons on a controller together with a mouse.
This is now fixed.

I still can't reproduce:

press & hold the Cue button > move outside > release

Does it still happen after the latest commit?

On which Qt version and OS did you test?

@ronso0
Copy link
Member

ronso0 commented Nov 25, 2020

The hotcue buttons seem to be fixed (tested with GUI and keyboard).

press & hold the Cue button > move outside > release

This still happens. When I right-click outside the play position jumps back to the Cue point and keeps playing. Though with the last commit it switches to normal play mode, clicking Play stops the track.
(this is also what happened earlier, it did not get stuck in preview with Cue)

this is with Ubuntu Studio 20.04, Qt 5.12.8

@daschuer
Copy link
Member Author

I cant confirm this with plain Ubuntu 20.04, Qt 5.12.8 and Unity.

@daschuer
Copy link
Member Author

Ah, now I got it, it is only the cue button affected.

@ronso0
Copy link
Member

ronso0 commented Nov 25, 2020

Yes, only the main Cue.

@daschuer
Copy link
Member Author

It is the "intended" right click action cue_gotoandstop. The stop part is the same as pressing the play button for latching.
I am heading towards to remove the goto part during review and just latch play. This allows to continue playing with only a mouse.
I think I can remember that this was working in the past.
The alternative would be to just end preview and perform cue_gotoandstop.
What do you think?

@ronso0
Copy link
Member

ronso0 commented Nov 26, 2020

the Cue button is not stuck so if the goto part was removed that indeed be not a bug but a feature :)
We should make it discoverable. Can we cram that info into the tooltip or is it sufficient to add it to the manual?
Right-click while previewing with pressed Cue button will latch the play button.

@daschuer
Copy link
Member Author

Done

@github-actions github-actions bot added the build label Nov 26, 2020
@ronso0
Copy link
Member

ronso0 commented Nov 26, 2020

Nice, works nicely with mouse only, didn't test other situations with controller.

there are conflicts now (#3370)

@daschuer
Copy link
Member Author

Ups, I messed it up with other tests. Fixed.

@uklotzde
Copy link
Contributor

@daschuer Please fix the conflicts.
@ronso0 Then this is ready I guess?

@uklotzde uklotzde added this to the 2.3.0 milestone Nov 27, 2020
@ronso0
Copy link
Member

ronso0 commented Nov 28, 2020

Yup, ready to go IMO.

@daschuer
Copy link
Member Author

Done.

@uklotzde
Copy link
Contributor

Thank you! LGTM

@uklotzde uklotzde merged commit 097df28 into mixxxdj:2.3 Nov 28, 2020
daschuer added a commit to daschuer/mixxx that referenced this pull request Jan 10, 2021
daschuer added a commit to daschuer/mixxx that referenced this pull request Jan 10, 2021
@daschuer daschuer mentioned this pull request Jan 10, 2021
@daschuer daschuer deleted the stuck_hotcue branch September 26, 2021 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants